Skip to content

[#1226] Remove 19+ toggle from NavBar, restore FilterBar checkbox#1227

Merged
realproject7 merged 1 commit into
mainfrom
task/1226-remove-navbar-19plus
May 18, 2026
Merged

[#1226] Remove 19+ toggle from NavBar, restore FilterBar checkbox#1227
realproject7 merged 1 commit into
mainfrom
task/1226-remove-navbar-19plus

Conversation

@realproject7
Copy link
Copy Markdown
Owner

@realproject7 realproject7 commented May 18, 2026

Summary

  • Removes the round 19+ icon from NavBar (and useNsfwPreference import) — NavBar is back to logo + nav links + wallet
  • Adds a 19+ checkbox to FilterBar's desktop filter row, after the Language dropdown
  • Mobile filter sheet already had the 19+ toggle — no changes needed there
  • All other 19+ renames, NsfwBadge on story cards, and profile page filtering are preserved

Test plan

  • NavBar has no 19+ icon on desktop or mobile
  • FilterBar desktop row shows 19+ checkbox after Language dropdown
  • Checking the box adds ?nsfw=1 to the URL
  • Unchecking removes it
  • Mobile filter sheet still has "Show 19+ content" toggle
  • Active chip "19+" appears on mobile when enabled
  • NsfwBadge still renders on NSFW story cards

Fixes #1226

🤖 Generated with Claude Code

@realproject7 @claude
[#1226] Remove 19+ toggle from NavBar, add checkbox to FilterBar
161de78 
- Remove round 19+ icon and useNsfwPreference from NavBar
- Restore NavBar to logo + nav links + wallet only
- Add 19+ checkbox to FilterBar desktop row (after Language dropdown)
- Mobile filter sheet already has 19+ toggle — no changes needed
- Bump version to 1.29.3

Fixes #1226

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
plotlink Ignored Ignored May 18, 2026 5:38am

Request Review

@realproject7
Copy link
Copy Markdown
Owner Author

realproject7 commented May 18, 2026

Reviewer 2 — APPROVE

Clean reversal of #1225 + correct FilterBar restoration.

  • NavBar: 19+ toggle, useNsfwPreference import, and URL-sync logic fully removed. Logo section restored to original structure. No leftover imports or dead code.
  • FilterBar: Checkbox correctly uses existing nsfw variable (from showNsfw prop) and navigate() which already accepts nsfw. Placement after Language dropdown is logical. Styling uses CSS custom properties consistent with surrounding filters.
  • Mobile: Filter sheet already had the toggle — correctly left untouched.
  • Net -21 lines. No issues found.

project7-interns
project7-interns approved these changes May 18, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE

Summary

The PR removes the NavBar 19+ icon and useNsfwPreference dependency from NavBar, then restores the 19+ checkbox in the desktop FilterBar row using the existing URL-param navigation path. The preserved 19+ behavior outside NavBar remains in scope, and CI is passing.

Findings

  • None.

Decision

Approve. The implementation satisfies issue #1226 acceptance criteria, and lint-and-typecheck plus e2e both pass.

@realproject7 realproject7 merged commit 551ce21 into main May 18, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@project7-interns project7-interns project7-interns approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Fix] Remove 19+ toggle from NavBar, restore FilterBar checkbox

2 participants

@realproject7 @project7-interns